-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature #7
base: master
Are you sure you want to change the base?
Feature #7
Conversation
simple append css invert to html dom so if using tag <a> it will refresh otherwise <route-link>
components/Home.vue
Outdated
<div | ||
v-if="filteredList.length === 0" | ||
class="empty-list" | ||
> | ||
Ooops! Nothing here..🙈 | ||
落了片白茫茫大地真干净。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
建议不要直接 hardcode 中文,可以考虑支持下多语言,或者是直接用 $lang
判断
example/.vuepress/config.js
Outdated
}, | ||
plugins: [ | ||
'@vuepress/search', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
如果要加上 Search 插件,不能只给 example 加,要放到顶级的 index.js 里
参考 https://vuepress.vuejs.org/theme/writing-a-theme.html#apply-plugins
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Get it.
非常感谢你提交的一些新功能,我本地用过之后觉得很棒。 但在合并之前先提一些建议:
欢迎继续讨论。 |
感谢回复。 翻页或许是见仁见智的,首页列表过长(>50)会影响观感,作为配置项如何,这是否违背了 simple 的初衷? |
Thank you for making this theme.